docs: sync country coverage with Grid Switch Corridor List#534
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
Greptile SummaryThis PR syncs the country coverage documentation with the Grid Switch Corridor List by adding Egypt (EG) as a live Bank Transfer destination and updating the regional and headline country counts.
Confidence Score: 5/5Safe to merge — the only code change is inserting one table row and updating three count strings, all of which are internally consistent. The change is a small, self-contained documentation update. Egypt is placed correctly in alphabetical order, the MEA and headline counts are consistent with the full table (32+15+10+5=62), and no logic or API behaviour is touched. The open question about China's live status and the PR description's arithmetic are worth resolving but do not block the Egypt addition itself. mintlify/snippets/country-support.mdx — the CN row warrants a second look against the spreadsheet source of truth.
|
| Filename | Overview |
|---|---|
| mintlify/snippets/country-support.mdx | Adds Egypt (EG) to the Live countries table, bumps MEA count 14→15, and updates headline count 60→62. The regional math (32+15+10+5=62) is internally consistent, but the PR description misstates the bump as 60→61 and APAC as 9; the CN entry in the Live table may conflict with the spreadsheet source of truth. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Grid Switch Corridor List\nSpreadsheet] -->|Status = Live filter| B[Live Destinations]
A -->|Public Roadmap = Yes\nAND Status ≠ Live| C[Coming Soon]
B -->|Egypt EG added| D[country-support.mdx\nLive Table]
C -->|CA, HK, KR unchanged| E[country-support.mdx\nComing Soon Section]
D --> F[Headline: 62 Countries\n32 EU + 15 MEA + 10 APAC + 5 Americas]
style B fill:#d4edda,stroke:#28a745
style C fill:#fff3cd,stroke:#ffc107
Prompt To Fix All With AI
Fix the following 2 code review issues. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 2
mintlify/snippets/country-support.mdx:10-12
**Headline jumps by 2, not 1**
The PR description says "Bump headline count: 60 → 61" and lists the regional totals as "32 + 15 + 9 + 5 = 61," but the actual code changes both strings from **60 to 62**. Counting the table rows before this PR yields 61 countries (the old headline of 60 was already under by one), so the final 62 is mathematically correct (32 + 15 + 10 + 5 = 62, with Asia-Pacific at 10, not 9). The code itself is fine, but the PR description arithmetic is inconsistent — worth double-checking that the pre-existing off-by-one was intentional to fix here rather than a copy-paste slip.
### Issue 2 of 2
mintlify/snippets/country-support.mdx:27
**China appears in Live table but PR description classifies it as Coming Soon**
The PR description states "Coming Soon (CA, CN, HK, KR) already matches the spreadsheet filter (Public Roadmap = Yes AND Status ≠ Live)," implying the spreadsheet marks `CN` as not-yet-live. However, `CN` is currently in the Live countries table (line 27) and is absent from the Coming Soon section (which only lists CA, HK, KR). If the spreadsheet's source of truth has China as non-live, this row should be moved to the Coming Soon section rather than remaining as a live corridor.
Reviews (3): Last reviewed commit: "docs: sync country coverage with Grid Sw..." | Re-trigger Greptile
e5e09e6 to
fe747bc
Compare
Add Egypt as a Live destination (Thunes, Bank Transfer) and update counts: 62 total Live destinations (Middle East and Africa 14 -> 15). Coming Soon (Canada, Hong Kong, South Korea) unchanged; China is now Live and already listed. https://claude.ai/code/session_01QjQtezWVr4v9wdDtK2tFTJ
fe747bc to
4912bab
Compare
Summary
Sync
mintlify/snippets/country-support.mdxwith the Grid Switch Corridor List source of truth (tab: Grid Switch Corridor List).Bank Transfer).FeatureCardtags + "Send payments to N countries" copy). The published table actually listed 61 rows while the copy still said 60; this brings copy, tag, table, and regional totals into agreement.Live derivation
Live destinations = rows with Status = "Live" (column J), deduped on ISO code. China (CN) stays a Live destination: it has a Live corridor (Thunes) as well as a Scoping row with Public Roadmap = Yes (Tazapay), and the dedupe rule prefers the Live row — so China remains in the main table and is not listed under Coming Soon. Egypt was the only net-new Live destination vs. the current published table.
Test plan
make lint-markdown(no OpenAPI changes, no rebundle needed)mint dev